Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create REAEME.md #24

Closed
wants to merge 3 commits into from
Closed

Create REAEME.md #24

wants to merge 3 commits into from

Conversation

aiclaudev
Copy link
Contributor

Co-Authored-By: SEONG PIL JU [email protected]

What does this PR do? Please write it down in detail.

  • What was the previous problem ?
    There is no README.md in euphony-io/soundless-check-in.

  • How did you solve it?
    Through discussing, Add Reason for making this application, Run the app, Report Issues, Library category.

Link to the related issue

Issue #2

Is there anything you have tried to improve but failed?

I wanted to add 'What is next' category, but i don't know what to write in this category.

Co-Authored-By: SEONG PIL JU <[email protected]>
Copy link
Member

@jopopcorn jopopcorn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You've done a great job writing README!

README.md Outdated
Comment on lines 5 to 6
This is a check-in application that identifies visitors, especially effective during the COVID-19.
This application stores below informations in database
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to apply the enter one more time.

Suggested change
This is a check-in application that identifies visitors, especially effective during the COVID-19.
This application stores below informations in database
This is a check-in application that identifies visitors, especially effective during the COVID-19.
This application stores below informations in database.

README.md Outdated
Comment on lines 42 to 44
It is based on Euphony library.
Official Facebook Page : https://www.facebook.com/euphonyproject
Official Library Site : https://dev.jbear.co/euphony
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please apply Enter to this part as well.

Suggested change
It is based on Euphony library.
Official Facebook Page : https://www.facebook.com/euphonyproject
Official Library Site : https://dev.jbear.co/euphony
It is based on Euphony library.
Official Facebook Page : https://www.facebook.com/euphonyproject
Official Library Site : https://dev.jbear.co/euphony

README.md Outdated
Comment on lines 37 to 38
To report a specific problem or feature request, [please open a new issue](https://github.com/euphony-io/soundless-check-in/issues/new/choose).
Your participation drives the development of the application!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please apply Enter to this part as well!

Suggested change
To report a specific problem or feature request, [please open a new issue](https://github.com/euphony-io/soundless-check-in/issues/new/choose).
Your participation drives the development of the application!
To report a specific problem or feature request, [please open a new issue](https://github.com/euphony-io/soundless-check-in/issues/new/choose).
Your participation drives the development of the application!

aiclaudev and others added 2 commits August 28, 2021 19:18
Co-Authored-By: SEONG PIL JU <[email protected]>
Add new line below 'This application stores below informations in database' sentence

Co-Authored-By: SEONG PIL JU <[email protected]>
@aiclaudev
Copy link
Contributor Author

@jopopcorn
Thank you!
I updated file reflecting your feedback :)

@jopopcorn
Copy link
Member

Great! Thanks your efforts 😄

Copy link
Member

@sujin9 sujin9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect! :)
And in 'What's next', i'd like to add phone verification service when user enter their phone number!

@aiclaudev
Copy link
Contributor Author

Perfect! :)
And in 'What's next', i'd like to add phone verification service when user enter their phone number!

It's so great idea !!
After the application is completed, I will add it when I organize it for the last time.
Thanks for your feedback :)

@jopopcorn
Copy link
Member

I would appreciate it if you could add a license when modifying README!


License

soundless-check-in is licensed under the Apache 2.0 license. See the LICENSE file for details.

@aiclaudev aiclaudev closed this Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants